-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: New user properties for Analytics [WPB-16121] #3890
base: develop
Are you sure you want to change the base?
Conversation
@borichellow looks like you are rolling back kalium to a previous commitish. This means that the PR's target branch (develop) is using a newer version of Kalium, and the changes in this PR will rollback Kalium to an older version.
Is this intentional? |
@borichellow looks like you are rolling back kalium to a previous commitish. This means that the PR's target branch (develop) is using a newer version of Kalium, and the changes in this PR will rollback Kalium to an older version.
Is this intentional? |
@borichellow looks like you are rolling back kalium to a previous commitish. This means that the PR's target branch (develop) is using a newer version of Kalium, and the changes in this PR will rollback Kalium to an older version.
Is this intentional? |
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3890 +/- ##
===========================================
+ Coverage 45.58% 45.61% +0.03%
===========================================
Files 491 491
Lines 17071 17079 +8
Branches 2849 2848 -1
===========================================
+ Hits 7781 7790 +9
Misses 8495 8495
+ Partials 795 794 -1
Continue to review full report in Codecov by Sentry.
|
Built wire-android-staging-compat-pr-3890.apk is available for download |
Built wire-android-dev-debug-pr-3890.apk is available for download |
What's new in this PR?
New user properties for Countly:
team_is_enterprise
,team_is_team
,team_team_id
,team_team_size
,user_contacts
.Solutions
Use new kalium UseCase for getting combined
AnalyticsContactsData
and provide that values to Countly.Waiting for wireapp/kalium#3312 be merged.